Skip to content

Conversation

@singhpk234
Copy link
Contributor

@singhpk234 singhpk234 commented Mar 24, 2022

@github-actions github-actions bot added the docs label Mar 24, 2022
Copy link
Contributor

@anuragmantri anuragmantri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks for making the change.

@jackye1995 - Are we still adding docs changes to the main Iceberg repo? I remember a discussion in dev list on moving this out.

### S3 Access Points

[Access Points](https://docs.aws.amazon.com/AmazonS3/latest/userguide/using-access-points.html) can be used for operating
s3 objects from the specified bucket, against which access-point was mapped. This is useful for multi-region access,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @singhpk234 for working on this. It is really helpful. Wondering if we can elaborate more on how multi-region access is enabled by this feature. I assume we just need to clarify that one access point can access multiple buckets cross-regions. For the below example, we can add a spark-sql config with the different bucket name and the same access point, so that user can understand better how it works.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume we just need to clarify that one access point can access multiple buckets cross-regions. For the below example, we can add a spark-sql config with the different bucket name and the same access point, so that user can understand better how it works.

Makes sense, have made the changes, Thanks @flyrain :) !!!

@singhpk234 singhpk234 force-pushed the singhpk/access_points/doc branch from ccbebc2 to b19661a Compare April 20, 2022 14:25
@singhpk234 singhpk234 force-pushed the singhpk/access_points/doc branch from b19661a to 504305f Compare April 21, 2022 03:06
@singhpk234 singhpk234 force-pushed the singhpk/access_points/doc branch from 504305f to 516826d Compare April 26, 2022 04:26
@singhpk234 singhpk234 force-pushed the singhpk/access_points/doc branch from 516826d to fb77e63 Compare April 26, 2022 04:29
@singhpk234 singhpk234 requested a review from jackye1995 May 1, 2022 15:59
Copy link
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the work!

@jackye1995 jackye1995 merged commit a722331 into apache:master May 24, 2022
@singhpk234 singhpk234 deleted the singhpk/access_points/doc branch May 24, 2022 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants